Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[CoreML ] ML Program more operators support [3/N] #22710

Merged
merged 20 commits into from
Nov 28, 2024
Merged

[CoreML ] ML Program more operators support [3/N] #22710

merged 20 commits into from
Nov 28, 2024

Conversation

wejoncy
Copy link
Contributor

@wejoncy wejoncy commented Nov 4, 2024 •
edited
Loading

Description

  • Erf
  • Round
  • Max
  • ReduceMax
  • ReduceMean
  • ReduceSum
  • Unsqueeze
  • Squeeze
  • Softmax

Motivation and Context

@wejoncy wejoncy requested a review from skottmckay November 4, 2024 08:33
@wejoncy wejoncy marked this pull request as ready for review November 4, 2024 08:33
@wejoncy wejoncy changed the title [CoreML ] ML Program more operators support [CoreML ] ML Program more operators support [3/N] Nov 4, 2024
@wejoncy wejoncy mentioned this pull request Nov 4, 2024
Closed
@yuygfgg yuygfgg mentioned this pull request Nov 5, 2024
Merged
skottmckay
skottmckay reviewed Nov 6, 2024
View reviewed changes
onnxruntime/core/providers/coreml/builders/impl/binary_op_builder.cc Outdated Show resolved Hide resolved
onnxruntime/core/providers/coreml/builders/impl/reduction_op_builder.cc Show resolved Hide resolved
onnxruntime/core/providers/coreml/builders/impl/reduction_op_builder.cc Outdated Show resolved Hide resolved
onnxruntime/core/providers/coreml/builders/impl/reduction_op_builder.cc Outdated Show resolved Hide resolved
onnxruntime/core/providers/coreml/builders/impl/shape_op_builder.cc Outdated Show resolved Hide resolved
onnxruntime/core/providers/coreml/builders/impl/squeeze_op_builder.cc Outdated Show resolved Hide resolved
onnxruntime/core/providers/coreml/builders/impl/squeeze_op_builder.cc Outdated Show resolved Hide resolved
onnxruntime/core/providers/coreml/builders/impl/squeeze_op_builder.cc Outdated Show resolved Hide resolved
onnxruntime/core/providers/coreml/builders/impl/squeeze_op_builder.cc Outdated Show resolved Hide resolved
onnxruntime/core/providers/coreml/builders/impl/softmax_op_builder.cc Outdated Show resolved Hide resolved
@wejoncy wejoncy requested a review from skottmckay November 20, 2024 09:11
wejoncy and others added 2 commits November 22, 2024 10:36
@wejoncy @github-actions
Update onnxruntime/core/providers/coreml/builders/impl/binary_op_buil…
d3e652d 
…der.cc

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@wejoncy
simplify input_rank
3893f50
skottmckay
skottmckay previously approved these changes Nov 25, 2024
View reviewed changes
@wejoncy wejoncy requested a review from skottmckay November 27, 2024 01:28
@wejoncy wejoncy merged commit a24723d into main Nov 28, 2024
95 checks passed
@wejoncy wejoncy deleted the jicwen/coreml_ops branch November 28, 2024 01:21
guschmue pushed a commit that referenced this pull request Dec 2, 2024
@wejoncy @skottmckay
@github-actions @guschmue
[CoreML ] ML Program more operators support [3/N] (#22710)
ec12e03 
### Description
- Erf
- Round
- Max
- ReduceMax
- ReduceMean
- ReduceSum
- Unsqueeze
- Squeeze
- Softmax



### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->

---------

Co-authored-by: Scott McKay <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
ankitm3k pushed a commit to intel/onnxruntime that referenced this pull request Dec 11, 2024
@wejoncy @skottmckay
@github-actions @ankitm3k
[CoreML ] ML Program more operators support [3/N] (microsoft#22710)
00a4e37 
### Description
- Erf
- Round
- Max
- ReduceMax
- ReduceMean
- ReduceSum
- Unsqueeze
- Squeeze
- Softmax



### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->

---------

Co-authored-by: Scott McKay <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
ankitm3k pushed a commit to intel/onnxruntime that referenced this pull request Dec 11, 2024
@wejoncy @skottmckay
@github-actions @ankitm3k
[CoreML ] ML Program more operators support [3/N] (microsoft#22710)
7558609 
### Description
- Erf
- Round
- Max
- ReduceMax
- ReduceMean
- ReduceSum
- Unsqueeze
- Squeeze
- Softmax



### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->

---------

Co-authored-by: Scott McKay <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
ankitm3k pushed a commit to intel/onnxruntime that referenced this pull request Dec 11, 2024
@wejoncy @skottmckay
@github-actions @ankitm3k
[CoreML ] ML Program more operators support [3/N] (microsoft#22710)
14950cc 
### Description
- Erf
- Round
- Max
- ReduceMax
- ReduceMean
- ReduceSum
- Unsqueeze
- Squeeze
- Softmax



### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->

---------

Co-authored-by: Scott McKay <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments

@skottmckay skottmckay skottmckay approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wejoncy @skottmckay